Skip to content

[BUGFIX] table go-sdk: fix Density constant values#653

Merged
jgbernalp merged 1 commit into
perses:mainfrom
mrIncompetent:fix/table-sdk-density-constants
May 13, 2026
Merged

[BUGFIX] table go-sdk: fix Density constant values#653
jgbernalp merged 1 commit into
perses:mainfrom
mrIncompetent:fix/table-sdk-density-constants

Conversation

@mrIncompetent
Copy link
Copy Markdown
Contributor

Description

The Density constants in the table plugin's Go SDK held values the cue schema rejects ("auto", "always"), so any dashboard built with table.WithDensity(table.CompactDensity) or table.StandardDensity failed to load against the schema's "compact" | "standard" | "comfortable" disjunction. Aligned the constants with the schema and added the missing ComfortableDensity.

Investigated and authored with Claude Code.

Checklist

  • Pull request has a descriptive title and context useful to a reviewer.
  • Pull request title follows the [<catalog_entry>] <commit message> naming convention.
  • All commits have DCO signoffs.

Signed-off-by: Henrik Schmidt <mrIncompetent@users.noreply.github.com>
@mrIncompetent mrIncompetent requested a review from Nexucis as a code owner May 12, 2026 19:10
@jgbernalp jgbernalp enabled auto-merge May 13, 2026 16:56
@jgbernalp jgbernalp added this pull request to the merge queue May 13, 2026
Merged via the queue into perses:main with commit 898d17c May 13, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants